Skip to content

Use {{vscode}}/{{cli}} conditional blocks for platform-specific onboarding in paw-status#294

Merged
lossyrob merged 3 commits intofix/273-paw-about-uxfrom
copilot/sub-pr-290-again
Mar 17, 2026
Merged

Use {{vscode}}/{{cli}} conditional blocks for platform-specific onboarding in paw-status#294
lossyrob merged 3 commits intofix/273-paw-about-uxfrom
copilot/sub-pr-290-again

Conversation

Copy link
Contributor

Copilot AI commented Mar 15, 2026

The "get started" instructions added to paw-status/SKILL.md's Help Mode section listed both CLI and VS Code steps inline, instead of using the conditional template blocks the rest of the codebase uses.

Changes

  • skills/paw-status/SKILL.md: Replace platform-named bullet points with {{#cli}}/{{#vscode}} conditional blocks so each environment only sees its relevant instructions.

Before:

- If the user also asks how to get started, tailor the answer by platform:
  - **GitHub Copilot CLI**: install via `copilot plugin install ...`
  - **VS Code**: install the extension, then run `PAW: New PAW Workflow`.

After:

- If the user also asks how to get started:
{{#cli}}
  - Install via `copilot plugin install ...`
{{/cli}}
{{#vscode}}
  - Install the extension, then run `PAW: New PAW Workflow`.
{{/vscode}}

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI changed the title [WIP] [WIP] Address feedback on using vscode and cli templates for prompts Use {{vscode}}/{{cli}} conditional blocks for platform-specific onboarding in paw-status Mar 15, 2026
Copilot AI requested a review from lossyrob March 15, 2026 02:25
- Install via `copilot plugin install lossyrob/phased-agent-workflow` or `npx @paw-workflow/cli install copilot`, then start with `copilot --agent PAW` (or `/agent PAW` in an existing session).
{{/cli}}
{{#vscode}}
- Install the extension, then run `PAW: New PAW Workflow`.
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since the user will be using a skill that requires an installation, it makes less sense to have install guidance here. Just have the workflow init guidance. Right for vscode, for cli it should be selecting the PAW custom agent and typing, "start a new paw workflow" or something like that

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done in 74ca643. Removed the install guidance and updated both blocks to workflow init only:

  • CLI: "Select the PAW custom agent (copilot --agent PAW or /agent PAW) and say 'start a new PAW workflow'."
  • VS Code: "Run PAW: New PAW Workflow."

Copilot AI requested a review from lossyrob March 16, 2026 14:40
@lossyrob lossyrob marked this pull request as ready for review March 17, 2026 15:35
@lossyrob lossyrob merged commit 1ef340c into fix/273-paw-about-ux Mar 17, 2026
@lossyrob lossyrob deleted the copilot/sub-pr-290-again branch March 17, 2026 15:35
lossyrob added a commit that referenced this pull request Mar 17, 2026
* Plan PAW about UX fix

Co-authored-by: Copilot <[email protected]>

* Ground PAW onboarding in project docs

Co-authored-by: Copilot <[email protected]>

* Stop tracking PAW artifacts for paw-about-ux

Co-authored-by: Copilot <[email protected]>

* Fix paw-status onboarding: replace local doc refs with documentation website URLs (#293)

* Use {{vscode}}/{{cli}} conditional blocks for platform-specific onboarding in paw-status (#294)

* Initial plan

* Use conditional blocks for platform-specific content in paw-status

Co-authored-by: lossyrob <[email protected]>

* Remove install guidance; keep only workflow init instructions in paw-status help mode

Co-authored-by: lossyrob <[email protected]>

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: lossyrob <[email protected]>

---------

Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: lossyrob <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants